Skip to content

Conversation

@kevinjqliu
Copy link
Contributor

@kevinjqliu kevinjqliu commented Nov 4, 2025

Rationale for this change

openjdk:8-jre-slim is gone from dockerhub

#7 ERROR: docker.io/library/openjdk:8-jre-slim: not found

Let's use the apache/hive:4.0.0 image as base. Inspired by https://github.com/trinodb/docker-images/blob/master/testing/hive4.0-hive/Dockerfile

Are these changes tested?

Are there any user-facing changes?

Copy link
Contributor

@geruh geruh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Also had this failure in the uv #2601 PR this approach seems a lot cleaner. Works locally as well.

Copy link
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this @kevinjqliu and thanks for the quick review @geruh

@Fokko Fokko merged commit 9e16bc2 into apache:main Nov 4, 2025
8 checks passed
liurenjie1024 pushed a commit to apache/iceberg-rust that referenced this pull request Nov 4, 2025
## What changes are included in this PR?

We had some failures in the Pyiceberg repo with the hive docker file
here: apache/iceberg-python#2697, so I'm porting
this over here.

The HMS test Dockerfile was using a deprecated `openjdk:8-jre-slim` base
image that has very **recently** been removed from Docker Hub, causing
build failures:

```
#7 ERROR: docker.io/library/openjdk:8-jre-slim: not found
```

Simplified the Dockerfile to use apache/hive:3.1.3 as the base image
directly, also eliminating the multi-stage build pattern. Which removes
the dependency on a deprecated OpenJDK image and will use what's
included in hive, and maintains the same functionality for HMS
integration tests.

Inspired by
[](https://github.com/trinodb/docker-images/blob/master/testing/hive4.0-hive/Dockerfile)<https://github.com/trinodb/docker-images/blob/master/testing/hive4.0-hive/Dockerfile>
@kevinjqliu kevinjqliu deleted the kevinjqliu/replace-openjdk-8 branch November 4, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants